Increase timeout for `check_rule_active` from 60s to 120s by arista-nwolfe · Pull Request #22260 · sonic-net/sonic-mgmt

arista-nwolfe · 2026-02-05T19:33:02Z

We've seen that acl rule installation can take a while as discussed in #21988

This check_rule_active/check_rule_counters code was added in #21384
but it never worked on DNX due to unsupported SAI calls: sonic-net/sonic-buildimage#24665

Now that we're working around this issue with:
#22237
We're starting to run into timeouts waiting for ACL rules: Failed: Acl rule counters are not ready

Here are the logs from a failing run where you can see the time it takes for the last ACL rule to be installed takes more than 60s:

2026 Feb  5 03:44:23.106324 ctn101 INFO python[509061]: ansible-ansible.legacy.command Invoked with _raw_params=acl-loader update full /tmp/everflow/acl-erspan.json --table_name EVERFLOW_EGRESSV6 --session_name test_session_1 --mirror_stage egress
2026 Feb  5 03:44:27.267227 ctn101 INFO python[509601]: ansible-ansible.legacy.command Invoked with _raw_params=config load -y /tmp/everflow/acl_config.json

...

2026 Feb  5 03:46:22.380011 ctn101 NOTICE swss#orchagent: :- add: Successfully created ACL rule RULE_9 in table EVERFLOW_EGRESSV6

We're also increasing the timeout from 60s to 120s for the acl rules in the acl/ package tests here:
#21760

Type of change

Back port request

mssonicbld · 2026-02-05T19:33:10Z

/azp run

azure-pipelines · 2026-02-05T19:33:17Z

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

We've seen that acl rule installation can take a while as discussed in sonic-net#21988 This check_rule_active/check_rule_counters code was added in sonic-net#21384 but it never worked on DNX due to unsupported SAI calls: sonic-net/sonic-buildimage#24665 Now that we're working around this issue with: sonic-net#22237 We're starting to run into timeouts waiting for ACL rules: Failed: Acl rule counters are not ready Signed-off-by: Nathan Wolfe <[email protected]>

mssonicbld · 2026-02-05T19:35:07Z

/azp run

azure-pipelines · 2026-02-05T19:35:14Z

Azure Pipelines will not run the associated pipelines, because the pull request was updated after the run command was issued. Review the pull request again and issue a new run command.

eswaran-nexthop · 2026-02-09T21:18:05Z

tests/everflow/everflow_test_utilities.py


        if duthost.facts['asic_type'] != 'vs':
-            pytest_assert(wait_until(60, 2, 0, self.check_rule_active, duthost, table_name),
+            pytest_assert(wait_until(120, 2, 0, self.check_rule_active, duthost, table_name),


Actually, I had trouble getting the test pass even at 120s consistently, so I had to increase this to 150s to get it to pass. Are you seeing consistent success with q3d with this timeout?

We've only ran it a handful of times but all passed. I'm fine with bumping this to 150s as I was also seeing the timings from the logs were pretty close to 120s.

Updated to 150s

Signed-off-by: Nathan Wolfe <[email protected]>

mssonicbld · 2026-02-09T21:21:42Z

/azp run

azure-pipelines · 2026-02-09T21:22:00Z

Azure Pipelines successfully started running 1 pipeline(s).

abdosi · 2026-02-18T18:58:13Z

@arista-nwolfe : Please create Issue with slowness of rule insertion

…22260) * Increase timeout for check_rule_active from 60s to 120s We've seen that acl rule installation can take a while as discussed in sonic-net#21988 This check_rule_active/check_rule_counters code was added in sonic-net#21384 but it never worked on DNX due to unsupported SAI calls: sonic-net/sonic-buildimage#24665 Now that we're working around this issue with: sonic-net#22237 We're starting to run into timeouts waiting for ACL rules: Failed: Acl rule counters are not ready Signed-off-by: Nathan Wolfe <[email protected]> * Bump up from 120s to 150s as timeouts are still seen Signed-off-by: Nathan Wolfe <[email protected]> --------- Signed-off-by: Nathan Wolfe <[email protected]> Signed-off-by: Zhuohui Tan <[email protected]>

arista-nwolfe · 2026-02-28T00:12:52Z

@arista-nwolfe : Please create Issue with slowness of rule insertion

Requested issue: sonic-net/sonic-buildimage#25565

…22260) * Increase timeout for check_rule_active from 60s to 120s We've seen that acl rule installation can take a while as discussed in sonic-net#21988 This check_rule_active/check_rule_counters code was added in sonic-net#21384 but it never worked on DNX due to unsupported SAI calls: sonic-net/sonic-buildimage#24665 Now that we're working around this issue with: sonic-net#22237 We're starting to run into timeouts waiting for ACL rules: Failed: Acl rule counters are not ready Signed-off-by: Nathan Wolfe <[email protected]> * Bump up from 120s to 150s as timeouts are still seen Signed-off-by: Nathan Wolfe <[email protected]> --------- Signed-off-by: Nathan Wolfe <[email protected]> Signed-off-by: Mihut Aronovici <[email protected]>

…22260) * Increase timeout for check_rule_active from 60s to 120s We've seen that acl rule installation can take a while as discussed in sonic-net#21988 This check_rule_active/check_rule_counters code was added in sonic-net#21384 but it never worked on DNX due to unsupported SAI calls: sonic-net/sonic-buildimage#24665 Now that we're working around this issue with: sonic-net#22237 We're starting to run into timeouts waiting for ACL rules: Failed: Acl rule counters are not ready Signed-off-by: Nathan Wolfe <[email protected]> * Bump up from 120s to 150s as timeouts are still seen Signed-off-by: Nathan Wolfe <[email protected]> --------- Signed-off-by: Nathan Wolfe <[email protected]> Signed-off-by: Ravali Yeluri (WIPRO LIMITED) <[email protected]>

…22260) * Increase timeout for check_rule_active from 60s to 120s We've seen that acl rule installation can take a while as discussed in sonic-net#21988 This check_rule_active/check_rule_counters code was added in sonic-net#21384 but it never worked on DNX due to unsupported SAI calls: sonic-net/sonic-buildimage#24665 Now that we're working around this issue with: sonic-net#22237 We're starting to run into timeouts waiting for ACL rules: Failed: Acl rule counters are not ready Signed-off-by: Nathan Wolfe <[email protected]> * Bump up from 120s to 150s as timeouts are still seen Signed-off-by: Nathan Wolfe <[email protected]> --------- Signed-off-by: Nathan Wolfe <[email protected]> Signed-off-by: Abhishek <[email protected]>

…22260) * Increase timeout for check_rule_active from 60s to 120s We've seen that acl rule installation can take a while as discussed in sonic-net#21988 This check_rule_active/check_rule_counters code was added in sonic-net#21384 but it never worked on DNX due to unsupported SAI calls: sonic-net/sonic-buildimage#24665 Now that we're working around this issue with: sonic-net#22237 We're starting to run into timeouts waiting for ACL rules: Failed: Acl rule counters are not ready Signed-off-by: Nathan Wolfe <[email protected]> * Bump up from 120s to 150s as timeouts are still seen Signed-off-by: Nathan Wolfe <[email protected]> --------- Signed-off-by: Nathan Wolfe <[email protected]> Signed-off-by: Venkata Gouri Rajesh Etla <[email protected]>

github-actions bot requested review from StormLiangMS, abdosi and xwjiang-ms February 5, 2026 19:33

arista-nwolfe added this to SONiC Chassis Feb 5, 2026

arista-nwolfe force-pushed the master-everflow-ipv6-rule-timeout-increase branch from 9c8ab96 to db1856b Compare February 5, 2026 19:35

arista-nwolfe mentioned this pull request Feb 5, 2026

[Chassis] Added multi Asics support to everflow Ipv6 #22237

Merged

10 tasks

eswaran-nexthop reviewed Feb 9, 2026

View reviewed changes

Bump up from 120s to 150s as timeouts are still seen

3b23f93

Signed-off-by: Nathan Wolfe <[email protected]>

abdosi approved these changes Feb 18, 2026

View reviewed changes

abdosi merged commit 90cc07c into sonic-net:master Feb 18, 2026
18 checks passed

github-project-automation bot moved this to Done in SONiC Chassis Feb 18, 2026

davidm-arista added the Request for 202511 branch Request to backport a change to 202511 branch label Mar 3, 2026

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Increase timeout for `check_rule_active` from 60s to 120s#22260

Increase timeout for `check_rule_active` from 60s to 120s#22260
abdosi merged 2 commits intosonic-net:masterfrom
arista-nwolfe:master-everflow-ipv6-rule-timeout-increase

arista-nwolfe commented Feb 5, 2026 •

edited

Loading

Uh oh!

mssonicbld commented Feb 5, 2026

Uh oh!

azure-pipelines bot commented Feb 5, 2026

Uh oh!

mssonicbld commented Feb 5, 2026

Uh oh!

azure-pipelines bot commented Feb 5, 2026

Uh oh!

eswaran-nexthop Feb 9, 2026

Uh oh!

arista-nwolfe Feb 9, 2026

Uh oh!

arista-nwolfe Feb 9, 2026

Uh oh!

mssonicbld commented Feb 9, 2026

Uh oh!

azure-pipelines bot commented Feb 9, 2026

Uh oh!

abdosi commented Feb 18, 2026

Uh oh!

Uh oh!

arista-nwolfe commented Feb 28, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

Conversation

arista-nwolfe commented Feb 5, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Type of change

Back port request

Uh oh!

mssonicbld commented Feb 5, 2026

Uh oh!

azure-pipelines bot commented Feb 5, 2026

Uh oh!

mssonicbld commented Feb 5, 2026

Uh oh!

azure-pipelines bot commented Feb 5, 2026

Uh oh!

eswaran-nexthop Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

arista-nwolfe Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

arista-nwolfe Feb 9, 2026

Choose a reason for hiding this comment

Uh oh!

mssonicbld commented Feb 9, 2026

Uh oh!

azure-pipelines bot commented Feb 9, 2026

Uh oh!

abdosi commented Feb 18, 2026

Uh oh!

Uh oh!

arista-nwolfe commented Feb 28, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

5 participants

arista-nwolfe commented Feb 5, 2026 •

edited

Loading